Repository navigation
Install on Claude Code, Codex and other agents, with a test for each - #3
Merged
Merged
Conversation
… for each, and every commit on main reaches plugin users
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
People install skills from Claude Code, Codex and other CLIs, so each route has to work and the README has to say how. This takes marketingskills' repo layout and README shape as the reference.
What changes
.agents/plugins/marketplace.jsonis Codex's own marketplace file. Codex already installed through its fallback to.claude-plugin/.claude plugin updatesaid "already at the latest version" and the new content never arrived. Without a version, Claude Code updated from one commit to the next.check-skills.mjsnow fails on a manifest with a version.scripts/test-installs.shinstalls into clean folders as plugins for Claude Code and Codex and throughnpx skillsfor 11 agents.--runalso asks each signed-in agent to use a skill..github/workflows/check.ymlrunscheck-skills.mjsand the install test on each pull request.CONTRIBUTING.md,AGENTS.mdandCLAUDE.mdhold what the README's "Changing the skills" section used to say, plus the repo's rules.Checked
From the published repo before this change, real sessions on Claude Code and Codex (both routes) and Cursor (
npx skills) found the skill, read its reference file and read OpenApps data. OpenCode and Amp weren't run because neither is signed in on this machine. Their files landed, and so did those for Gemini CLI, GitHub Copilot, Windsurf, Cline, Goose and Pi.I'll repeat this from GitHub after the merge, because the README's test table says it ran from there.